Feature/issue 211/create ci for frontend#213
Merged
Conversation
Add CI for frontend
Add unit tests for App.vue, color.js and language.js.
Add unit tests for Cookie.vue, Login.vue and MainView.vue
Add unit tests for HistorySidebar.vue, MainContent.vue and SettingSidebar.vue
Siyuan106
added
Frontend
Siyuan106
requested review from
GerardChabaBristol,
Liuwf4319,
RainBOY-ZZX,
vladislav-k1 and
yi23484
Liuwf4319
previously approved these changes
Mar 11, 2025
Contributor
Liuwf4319
left a comment
Liuwf4319
left a comment
There was a problem hiding this comment.
Highly effective test files. Good work
GerardChabaBristol
requested changes
Mar 11, 2025
Collaborator
GerardChabaBristol
left a comment
GerardChabaBristol
left a comment
There was a problem hiding this comment.
The GitHub Action ran successfully and the tests look good, however, I believe Russian isn't included in the tests in language.spec.js.
Add test for Russian translation in language.spec.js.
GerardChabaBristol
approved these changes
Mar 11, 2025
Collaborator
GerardChabaBristol
left a comment
GerardChabaBristol
left a comment
There was a problem hiding this comment.
Great, all good now.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue(s):
#211
Type of change: (choose required ones)
Description:
The front-end CI process automates testing and building for the AI learning tool's Vue.js application. It runs Jest unit tests to verify component interactions, checks for JavaScript syntax errors, and builds the production version. This process utilizes the Vue 3, Jest, Axios, and Vue Router technology stack to ensure code quality and prevent regressions when developing new features.
Additional context:
N/A.
Testing instructions:
Run "npm run test:unit" in frontend directory to start the test.